Skip to content

ci: Update release workflow for npm OIDC authentication and bump up node version#273

Merged
jaissica12 merged 3 commits intomainfrom
ci/SDKE-683-add-npm-oidc-release
Feb 4, 2026
Merged

ci: Update release workflow for npm OIDC authentication and bump up node version#273
jaissica12 merged 3 commits intomainfrom
ci/SDKE-683-add-npm-oidc-release

Conversation

@jaissica12
Copy link
Contributor

@jaissica12 jaissica12 commented Jan 15, 2026

Summary

  • npm now supports OIDC and requires an update to the release.yml and release.sh to support this
  • Bumped up below versions:
    • Node to LTS 24.x
Screenshot 2026-02-04 at 4 01 34 PM

Testing Plan

Master Issue

Closes https://go/j/SDKE-683

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the npm publishing workflow to use OIDC (OpenID Connect) authentication instead of token-based authentication, and upgrades Node.js to version 24 across all CI workflows.

Changes:

  • Migrated npm publishing from token-based authentication to OIDC trusted publishing
  • Upgraded Node.js from version 18 to version 24 across all GitHub Actions workflows
  • Added npm provenance configuration to package.json

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
release.sh Simplified script to use OIDC authentication, removing manual NPM_TOKEN handling
package.json Added publishConfig section with provenance settings for npm trusted publishing
.github/workflows/release.yml Added OIDC permissions, registry-url configuration, and Node.js 24 upgrade
.github/workflows/pull-request.yml Updated Node.js version from 18 to 24 across all jobs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@thomson-t thomson-t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jaissica12 can you rebase the changes on the latest main, please?
There was an issue with the iOS PR build due to the simulator version being unavailable, and it's been fixed now.

@jaissica12 jaissica12 force-pushed the ci/SDKE-683-add-npm-oidc-release branch from 90a6bb4 to 39f28f5 Compare February 4, 2026 20:59
remove duplicate access flag

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jaissica12 jaissica12 requested a review from thomson-t February 4, 2026 21:24
@jaissica12
Copy link
Contributor Author

@jaissica12 can you rebase the changes on the latest main, please? There was an issue with the iOS PR build due to the simulator version being unavailable, and it's been fixed now.

@thomson-t thanks for letting me know, rebased the changes on the latest main.

@jaissica12 jaissica12 merged commit a3cd982 into main Feb 4, 2026
8 checks passed
@jaissica12 jaissica12 deleted the ci/SDKE-683-add-npm-oidc-release branch February 4, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants